Skip to content

Query agent #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
May 6, 2025
Merged

Query agent #1

merged 3 commits into from
May 6, 2025

Conversation

@augustas1
Copy link
Collaborator

@augustas1 augustas1 commented Apr 29, 2025

Initial Query Agent pull request.

Depends on weaviate/typescript-client#292.

Has weaviate-client as a peer dependency and can be consumed like this:

# once published
npm install weaviate-client @weaviate/agents
import { connectToWeaviateCloud } from "weaviate-client";
import { QueryAgent } from "@weaviate/agents";

const client = await connectToWeaviateCloud("cluster-url", { authCredentials: "api-key" });
const queryAgent = new QueryAgent(client, ["Emails"]);

const response = await queryAgent.run("What is a common topic in the emails?");
response.display();

@github-advanced-security
Copy link

github-advanced-security bot commented Apr 29, 2025

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

orca-security-eu[bot]
orca-security-eu bot reviewed Apr 29, 2025
View reviewed changes
Copy link

@orca-security-eu orca-security-eu bot left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

@malgamves
Copy link

malgamves commented Apr 29, 2025

Everything looks good @augustas1 ! Do you have any tests planned for the repo?

malgamves
malgamves reviewed Apr 29, 2025
View reviewed changes
Copy link

@malgamves malgamves left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I couple files are missing, LICENSE, README etc, I could help with some of these as I work on examples if you'd like.

package.json
],
"author": "Weaviate",
"license": "SEE LICENSE IN LICENSE"
"license": "SEE LICENSE IN LICENSE",
Copy link

@malgamves malgamves Apr 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the LICENSE file is missing @augustas1

Copy link
Collaborator Author

@augustas1 augustas1 Apr 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's already on main

@augustas1
Copy link
Collaborator Author

augustas1 commented Apr 30, 2025

Yes, I'll add README, tests and CI later on. Didn't want to have a mega PR :).
Re examples, I think we usually have a link to the docs (once we have them).

@IchordeDionysos
Copy link

IchordeDionysos commented May 5, 2025

@augustas1 is there a rough timeline for when we could starting using the QueryAgent in TypeScript? 😍

tsmith023
tsmith023 reviewed May 5, 2025
View reviewed changes
Copy link

@tsmith023 tsmith023 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the CI, feel free to copy what you want/need from the TS client's workflow file

@augustas1
Copy link
Collaborator Author

augustas1 commented May 6, 2025 •
edited
Loading

@augustas1 is there a rough timeline for when we could starting using the QueryAgent in TypeScript? 😍

@IchordeDionysos I think Query Agent should be there in 1-2 weeks

@augustas1 augustas1 merged commit c534a27 into main May 6, 2025
3 checks passed
@augustas1 augustas1 deleted the query-agent branch May 6, 2025 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@orca-security-eu orca-security-eu[bot] orca-security-eu[bot] left review comments

@sebawita sebawita Awaiting requested review from sebawita

+2 more reviewers

@malgamves malgamves malgamves left review comments

@tsmith023 tsmith023 tsmith023 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@augustas1 @malgamves @IchordeDionysos @tsmith023